Skip to content

fix(web): keep pagination inside the card and make the logo visible - #18

Merged
vsilent merged 1 commit into
mainfrom
fix/ui-pagination-logo
Nov 2, 2026
Merged

vsilent merged 1 commit into
mainfrom
fix/ui-pagination-logo

Conversation

@vsilent

@vsilent vsilent commented Nov 2, 2026

Copy link
Copy Markdown
  • Alert pagination rendered one button per page: 2711 buttons at 27k alerts, in one unwrapped row spilling across the whole viewport. Windowed to first, last, current +/-1 with ellipses. The page also resets to 1 when the filter changes, so a filtered list cannot show an empty table behind stale page buttons.
  • The sidebar logo was loaded from a remote URL (blank whenever that host is unreachable) and is a black silhouette on a dark sidebar either way. Bundled as web/src/assets/logo.png with the webpack asset rule and TypeScript declarations, drawn inverted (white) via CSS.
  • Verified: web jest suite 54/54, production webpack build passes.

The alert list rendered one pagination button per page. At 27k alerts
that is 2711 buttons in one unwrapped row, spilling across the whole
viewport. Window the pages: first, last, current +/-1, ellipses. Also
reset to page 1 when the filter changes, so a filtered list cannot show
an empty table behind page buttons from the previous total.

The sidebar logo was loaded from a remote URL, blank whenever that host
is unreachable, and the artwork is a black silhouette on a dark sidebar
either way. Bundle it as web/src/assets/logo.png with the webpack asset
rule and TypeScript declarations, and draw it inverted (white) via CSS.

cargo test is unaffected; web jest suite 54/54 and the production
webpack build pass.
@vsilent
vsilent merged commit 396e197 into main Nov 2, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants